Repository navigation
fix: команды черновиков и получатели сертификатов Solution (R9/R11) - #854
Merged
Merged
Conversation
This was referenced Nov 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
При приемке frontend PR PROCOLLAB-github/procollab#517 обнаружены две ошибки серверных данных. В XLSX черновика капитан повторялся на листе команды: у Solution25 размер2, но строки84/84/341. Теперь капитан выводится один раз, роль принятого участника сохраняется.
Аудитория сертификатов «Только сдавшим» учитывала только Project и давала0 при трех сданных Solution. Для режима solution общий сервис получателей теперь использует капитана и принятых участников зафиксированного состава активных сданных Solution, пересекает их с регистрациями этой программы и исключает черновики/архив/ожидающие приглашения. Этот сервис используется счетчиками и фактической инициализацией очереди. Путь legacy_project сохранен.
Проверка: до исправления7 регрессий,6 FAIL; после —81 связанных Django/PostgreSQL тестов PASS (включая legacy сертификаты и native PDF), flake8 и diff-check PASS. Добавлены проверки XLSX, счетчиков API, состава очереди, snapshot против live drift, регистрации, program scope, архива, отсутствия fallback в Project и дедупликации. Миграций и изменений зависимостей нет.
Пользователь разрешил merge в dev и последующую реальную приемку. Merge — после успешного полного CI; production не затрагивается. Реальный повтор R9/R11 запланирован после штатного dev-деплоя.